Skip to content

fix(test): update AlpnHttpTest to use available port instead of fixed port 8080 that can be occupied by parallel tests#6824

Merged
joviegas merged 1 commit intomasterfrom
joviegas/alpn_h2_test
Mar 27, 2026
Merged

fix(test): update AlpnHttpTest to use available port instead of fixed port 8080 that can be occupied by parallel tests#6824
joviegas merged 1 commit intomasterfrom
joviegas/alpn_h2_test

Conversation

@joviegas
Copy link
Copy Markdown
Contributor

@joviegas joviegas commented Mar 26, 2026

Motivation and Context

  • AlpnHttpTest test intermittently failed with

java.lang.AssertionError: 

Expecting a throwable with cause being an instance of:
  software.amazon.awssdk.core.exception.SdkClientException
but was an instance of:
  software.amazon.awssdk.services.h2.model.H2Exception: Service returned HTTP status code 500 (Service: H2, Status Code: 500, Request ID: null) (SDK Attempt

This was because it got connected to port 8080 since parallel test were getting executed

Modifications

  • The fix is to get a unused port at the time of the test and close it immediately and then use that port for testing so that we dont pick up a port which is used by other parallel tests

Testing

  • Ran in loop and it passed

License

  • I confirm that this pull request can be released under the Apache 2 license

@joviegas
fix(test): update AlpnHttpTest to use available port instead of fixed…
8348b80 
… port 8080 that can be occupied by parallel tests
@joviegas joviegas requested a review from a team as a code owner March 26, 2026 23:02
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 26, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@joviegas joviegas added this pull request to the merge queue Mar 27, 2026
Merged via the queue into master with commit f755c3f Mar 27, 2026
39 of 41 checks passed
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 27, 2026

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@zoewangg zoewangg zoewangg approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@joviegas @zoewangg